-
-
Notifications
You must be signed in to change notification settings - Fork 127
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade Python version to 3.10 #811
Conversation
You missed my sneaky .gitlab-ci.yml test that references python:3.8 🐧 Aside: I can't leave a comment that isn't related to a diff in a review, right? Why not GitHub. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me aside from my comments. After changes are made, this is good to merge IMO.
dependencies/pip/requirements.in
Outdated
lxml | ||
pyxform==1.9.0 | ||
django-reversion<3.0.2 | ||
xlwt | ||
openpyxl | ||
celery>=4.0,<5.0 | ||
celery>=5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this any better than just celery
? It's saying use celery >= 5. So 6 would match that too. Any pip-compile --upgrade would pick the newest compatible version, so maybe unnecessary to specify a minimum version. I'm new to pip-tools so maybe I'm missing something.
Personally I prefer the ^ and ~ syntax to tie things to major and minor versions. Then liberally upgrading patch versions via pip-compile --upgrade
quite often to deliver security patches without major version changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, it did not want to upgrade to latest one on my environment. Indeed >=5
should not required to install the latest version.
I'll try again and change it if it works as expected.
e79f0c5
to
1fa30ad
Compare
1fa30ad
to
8b541b8
Compare
Description
Use Python 3.10 in docker image
Additional info
Changes:
Removed:
transifex-client
. Only support 3.9 max. Support Python 3.10 issue. We use GitHub Actions to compile translations.nose
Packages that cannot be upgraded now:
Related issues
kobotoolbox/kpi#3759